Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(bpdm): added connection to dependencies in readiness check for bpdm services #1164

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat(bpdm): added connection to dependencies in readiness check for bpdm services #1164

wants to merge 4 commits into from

Conversation

SujitMBRDI
Copy link
Contributor

@SujitMBRDI SujitMBRDI commented Dec 16, 2024 •
edited
Loading

Description

This pull request adds dependancy check for services in bpdm as below,

  • Upon startup the Pool is ready when a successful connection to the Orchestrator has been made.
  • Upon startup the Cleaning Service Dummy is ready when a successful connection to the Orchestrator has been made.
  • Upon startup the Gate is ready when a successful connection to the Orchestrator and Pool has been made.
  • Enhanced dependency readiness checks with a scheduler to verify connections to required services every 30 seconds and provided error log if any dependant service is not healthy.

Contributes to #1161

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

@SujitMBRDI SujitMBRDI added the enhancement New feature or request label Dec 16, 2024
@SujitMBRDI SujitMBRDI added this to the BPDM v6.3.0 / R25.03. milestone Dec 16, 2024
@SujitMBRDI SujitMBRDI requested a review from nicoprow December 16, 2024 13:20
@SujitMBRDI
Copy link
Contributor Author

@nicoprow, I have created draft PR to show my approach for issue 1161. Kindly have a look and provide me your opinion about it.

@SujitMBRDI SujitMBRDI force-pushed the feat/bpdm-connect-dependend branch from e37a5d0 to 8285474 Compare December 19, 2024 05:25
@SujitMBRDI SujitMBRDI marked this pull request as ready for review December 19, 2024 05:31
@SujitMBRDI
feat(bpdm): added connection to dependencies in readiness check for b…
91ce51d 
…pdm services

feat(bpdm): added connection to dependencies in readiness check for bpdm services

feat(bpdm): added connection to dependencies in readiness check for bpdm services
@SujitMBRDI SujitMBRDI force-pushed the feat/bpdm-connect-dependend branch from 8ed323c to 91ce51d Compare December 19, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicoprow nicoprow Awaiting requested review from nicoprow

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
BPDM v6.3.0 / R25.03.
Development

Successfully merging this pull request may close these issues.
1 participant
@SujitMBRDI